Skip to content

Conversation

@deviantintegral
Copy link
Owner

No description provided.

@deviantintegral deviantintegral changed the title Claude/refactor texttrait tests l ts cy test: remove redundant test/assertion in SharedFields/TextTraitTest Dec 18, 2025
@deviantintegral deviantintegral changed the base branch from main to test-cleanup December 18, 2025 00:47
Remove testSetTextIsPublic method as it is redundant. The remaining tests
(testSetTextWithNull, testHasText, testGetText) still achieve 100% code
coverage and kill all 4 mutations from infection mutation testing.
Remove testSetTextWithNull method as it is redundant. The remaining tests
(testHasText, testGetText) still achieve 100% code coverage and kill all 4
mutations from infection mutation testing.
Remove testHasText method as it is redundant. The remaining test
(testGetText) still achieves 100% code coverage and kills all 4
mutations from infection mutation testing.
Remove redundant assertNull assertion from testGetText method. The remaining
assertion still achieves 100% code coverage and kills all 4 mutations from
infection mutation testing.
@deviantintegral deviantintegral force-pushed the claude/refactor-texttrait-tests-LTsCy branch from 2ebc8f0 to 077665b Compare December 18, 2025 00:47
@deviantintegral deviantintegral marked this pull request as ready for review December 18, 2025 00:49
@deviantintegral deviantintegral merged commit b241d64 into test-cleanup Dec 18, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants